Skip to content

feat(help_pages): change includes to django-cotton components #5998

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

nazarbodak221
Copy link
Contributor

@nazarbodak221 nazarbodak221 commented Jul 17, 2025

Convert include files to components

For better code organization, these components were moved to a new cotton/ directory. This change improves modularity, makes the parent templates cleaner, and simplifies the reuse of UI elements.

Just let me know if you'd like different styles or colors – I'll change them.

Also sidebar to migration guide(includes) I'd like not to write in cotton, but write in migration guide directly. Say if I can do it.

Affected files:

court_endpoint.html
court_id_mappings.html
deprecation_notice.html
docket_endpoint.html
v3_deprecated_warning.html

Screenshots:

  • court_endpoint.html:
    image
  • court_id_mapping.html:
    image
  • deprecation_notice.html:
    image
  • docket_endpoint.html:
    image
  • v3_deprecated_warnings:
    image
  • components:
image image

This commit introduces the template for the new <c-callout> component
@nazarbodak221 nazarbodak221 marked this pull request as draft July 17, 2025 18:15
Add new CSS to style the alert components. It includes styles for different alert types
The new callout component is now documented under the 'Layout components' section.

The 'Other components' section has been renamed to 'Pure markup' for better clarity.
@nazarbodak221 nazarbodak221 marked this pull request as ready for review July 18, 2025 07:49
@nazarbodak221 nazarbodak221 marked this pull request as draft July 18, 2025 07:50
@nazarbodak221 nazarbodak221 marked this pull request as ready for review July 18, 2025 07:52
@nazarbodak221
Copy link
Contributor Author

@elisa-a-v @mlissner, could you please check if everything okay here? If so, I'll start implementing these cotton components in the other branches.

@elisa-a-v elisa-a-v self-assigned this Jul 18, 2025
@elisa-a-v elisa-a-v self-requested a review July 18, 2025 17:11
Copy link
Contributor

@elisa-a-v elisa-a-v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you please check if everything okay here?

Done, I tested locally and it all works nicely.

Thanks for the clean PR! LGTM :shipit:

@mlissner
Copy link
Member

Automerge set, thanks all!

@nazarbodak221, we have a new system to optimize our deployments. It will show up in your new PRs as a PR template:

https://github.com/freelawproject/courtlistener/blob/main/.github/PULL_REQUEST_TEMPLATE.md

The upshot is that it would be helpful if you added the needed labels to any open and future PRs you have. I did this one as an example:

image

@mlissner mlissner merged commit c302c7b into freelawproject:main Jul 18, 2025
9 checks passed
@nazarbodak221
Copy link
Contributor Author

Automerge set, thanks all!

@nazarbodak221, we have a new system to optimize our deployments. It will show up in your new PRs as a PR template:

https://github.com/freelawproject/courtlistener/blob/main/.github/PULL_REQUEST_TEMPLATE.md

The upshot is that it would be helpful if you added the needed labels to any open and future PRs you have. I did this one as an example:

image

That's awesome, but I thought I didn't have permission to do it.

image

@mlissner
Copy link
Member

Shoot, you may be right. That's OK, we'll be sure to get them labeled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants